Skip to content

fix: ruff check pre-commit fails due to "Failed to parse pyproject.toml"#661

Open
psyberck wants to merge 1 commit into
PrunaAI:mainfrom
psyberck:fix/pre-commit-ruff
Open

fix: ruff check pre-commit fails due to "Failed to parse pyproject.toml"#661
psyberck wants to merge 1 commit into
PrunaAI:mainfrom
psyberck:fix/pre-commit-ruff

Conversation

@psyberck
Copy link
Copy Markdown

@psyberck psyberck commented May 14, 2026

Upgrade ruff-pre-commit to the latest to avoid ruff check failure in every commit.

Description

Related Issue

Fixes #608

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactor (no functional change)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

Testing

  • I added or updated tests covering my changes
  • Existing tests pass locally (uv run pytest -m "cpu and not slow")

For full setup and testing instructions, see the Contributing Guide.

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code, especially for agent-assisted changes
  • I updated the documentation where necessary

Thanks for contributing to Pruna! We're excited to review your work.

New to contributing? Check out our Contributing Guide for everything you need to get started.

Note:

  • Draft PRs or PRs without a clear and detailed overview may be delayed.
  • Please mark your PR as Ready for Review and ensure the sections above are filled out.
  • Contributions that are entirely AI-generated without meaningful human review are discouraged.

@begumcig
Copy link
Copy Markdown
Member

begumcig commented Jun 1, 2026

Hi @psyberck, thank you so much for taking the time to add this!

Before merging, I’d like to make sure the new Ruff version doesn’t introduce any new errors. It looks like the linting tests are currently failing because of intel-pytorch-extension, which we recently removed from Pruna.

Could you please pull the latest changes from main, rebase your branch, and push it again? I think that should get the tests running properly, and then we should be good to merge the PR!

@psyberck psyberck force-pushed the fix/pre-commit-ruff branch from 4f70c78 to 5f6bb9b Compare June 1, 2026 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Pre-commit ruff hook fails: D420 rule unknown in pinned v0.14.6

2 participants